Merge pull request #2616 from HuijingHei/dup-osname

Remove dup "--os=OSNAME" line in output of "ostree admin config-diff --help"

Remove dup "--os=OSNAME" line in output of "ostree admin config-diff --help"

Small fix for https://github.com/ostreedev/ostree/issues/2594

Fix GNU style of two spaces for indent in https://github.com/ostreedev/ostree/pull/2612

Merge pull request #2612 from HuijingHei/deploy-karg-delete

Teach --karg-delete option to ostree admin deploy

Merge pull request #2614 from nikita-dubrovskii/issue-2867

s390x: ensure SecureExecution is enabled before sd-boot generation

s390x: ensure SecureExecution is enabled before sd-boot generation

Signed-off-by: Nikita Dubrovskii <nikita@linux.ibm.com>

Merge pull request #2580 from nikita-dubrovskii/sdboot_partition

s390x: generate sd-boot at its own partition

Teach --karg-delete option to ostree admin deploy

See https://github.com/ostreedev/ostree/issues/2586

s390x: generate sd-boot at its own partition

Signed-off-by: Nikita Dubrovskii <nikita@linux.ibm.com>

Merge pull request #2610 from cgwalters/ci-tweaks-2

ci: Move codestyle and min build to GH actions

ci: Move codestyle and min build to GH actions

Part of general momentum to leave heavy lifting (e.g. VM tests)
to Jenkins and Prow and use the cheaper/faster GH actions for
plain builds.

Merge pull request #2608 from cgwalters/fanalyzer-fixes

ci: Drop gcc `-fanalyzer` by default

ci: Drop all `runAsUser: 0` in builds

This triggers the new git warning; I think this is only
here because long ago we used coreos-assembler as a buildroot,
but that's long since fixed.

ci: Drop gcc `-fanalyzer` by default

There's a pile of new warnings in GCC 12 from F36, I dug into
them but most of them seem inscrutable - I am not seeing the bugs.

For now to unblock further work, stop using this flag.

sysroot: Strengthen several `g_return_if_fail` into `g_assert()`

Similar to prior changes; motivated by testing out an updated
`gcc -fanalyzer`.

sysroot: Quiet one gcc `-fanalyzer` warning

I don't think this can happen in practice, but add an assertion
to verify it.

Merge pull request #2606 from ostreedev/dependabot/submodules/libglnx-c59eb27

build(deps): bump libglnx from `c71f7ae` to `c59eb27`

build(deps): bump libglnx from `c71f7ae` to `c59eb27`

Bumps libglnx from `c71f7ae` to `c59eb27`.

---
updated-dependencies:
- dependency-name: libglnx
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #2603 from cgwalters/use-cargo-deny

ci: use cargo-deny

ci: use cargo-deny

Copied from https://github.com/ostreedev/ostree-rs-ext/pull/291

Part of unifying our CI.

Merge pull request #2575 from cgwalters/merge-ostree-rs

Merge ostree-rs into this repository

Merge pull request #2601 from cgwalters/expose-get-xattrs

Add APIs to get xattrs from disk

Merge pull request #2602 from cgwalters/fix-docs-ci

ci: Work around GH actions `container:` + git + security bug

rust-bindings: Remove some unused CI/test bits

We're not using Vagrant or Gitlab, and our container flow is
different.

ci: Move rust-bindings CI to toplevel

It should replace our stub one.

tests/inst: Add .gitignore

Need this now that it is it's own workspace.

ci: Work around GH actions `container:` + git + security bug

See https://github.com/actions/checkout/issues/760

tests/inst: Fix install rules for ostree-rs merger

`tests/inst` became its own workspace.

Add APIs to get xattrs from disk

I'm aiming to do some more work on the Rust side around `fsck`
like functionality, and this is a useful primitive.  There isn't
a great Rust crate for xattrs, and I think it's better to share this
code.

cfg.mk: Don't even look at rust-bindings/

It's really tempting to remove `make syntax-check`, it has very
very rarely found any real problems.

But anyways, just exclude all the binding code because it trips
up random problems we simply don't care about like mentions of
`O_NDELAY` in the `GLib-2.0.gir`.

Merge pull request #2600 from cgwalters/link-suse-ima

docs/ima: Also link the SUSE docs

docs/ima: Also link the SUSE docs

They have a lot of stuff there.

build-sys: Adjust for merge of ostree-rs

Fix up the paths for the crates now that the Rust bindings are in
`rust/`.

We can't today include the test suite because it depends on `ostree-rs-ext`
which would make everything circular.

(Building that now requires a separate `cd tests/inst && cargo build`)

Add a `repo()` accessor to `TransactionGuard`

I want to write APIs that *require* the caller to have set up
an ostree transaction.  It's natural to require passing a guard
to do so.  But then we want an accessor for the repo.

lib: Run `cargo fmt`

Prep for merge into ostree, where we want to run `cargo fmt` checks
in CI.

ostree: release 0.13.7

ostree: manually patch generated files

This manually adds a missing `ToGlibPtr` import, which seems to be result
of some bugs in `gir` code-generation.

ostree: refresh after gir bump

ostree/cargo: bump to ostree-sys 0.9.2

ostree-sys: release 0.9.2

ostree-sys: refresh after gir bump

gir-files: bump to v2022.2

repo: Add `query_file` API

The underlying `ostree_repo_load_file()` API has the caller pass
`NULL` for output arguments it doesn't want.  This isn't sanely
bindable in Rust - what the generator does is always request
all values, but maps them all to `Option<T>`.

The main cases are where a user wants either metadata, or both
metadata and content.  This API gives just metadata; it's a
bit more efficient as we don't need to open the file, and doesn't
require the caller to `unwrap()`.

Add manual bindings for MutableTree reading

I'm trying to debug a problem in ostree-rs-ext, and it's
handy to be able to do `dbg!(mtree.copy_files())`.

Release 0.13.6

Fast-track fix for `ostree_gpg_verify_result_get_all()`

This cherry picks just the changes from
https://github.com/ostreedev/ostree/pull/2537

We don't need to wait to respin a new ostree release just
for this.

Release 0.13.5

Add `COMMIT_META_CONTAINER_CMD` constant

Today we hardcode `/bin/bash` in
https://github.com/coreos/coreos-assembler/blob/2088d24884771093101d95f915c921505128ef76/src/cmd-build#L405

But that breaks the concept of a bidirectional bridge between
container image and ostree commit because this little bit of
knowledge is encoded at the buildsystem side.

This metadata key is intended to be written into an ostree commit,
and then we will use it automatically in `container encapsulate`.

The "source of truth" for this key will hence be able live in the same
place that's generating the ostree commit.

The more "proper" place for this is probably alongside the other
constants in the libostree core C code.  But that's tedious and
slow to release.  And Rust is the future.  And we've been slowly
adding more "core ostree" functionality here.

repo: Add two more cap-std APIs

Followup to the previous PR.  I realized now with `io_lifetimes`
we can offer a safe `dfd_borrow()` that *borrows* the file descriptor
for the repository.  (In contrast to the current `.dfd()` that returns
the raw version)

Building on that, add another API that re-acquires a `Dir` instance.
(In the future in theory we could optimize this more by knowing
 whether or not the repo was constructed via cap-std, and perhaps
 in theory synthesize a `&Dir` reference, but I don't think we
 need that now)

Add a `cap-std-apis` feature with open/create

I'm trying to make more use of `cap-std` in our stack, and
this will be a key enabling API.

Actually a notable side benefit of this is that we don't need
to teach the ostree C code itself to use `openat2`, we inherit
cap-std's setup.

All of the internal ostree code using the prior `openat()` should
continue to work.

I only did basic sanity checking of this; there may be bugs
in other APIs.

Release 0.13.4

repo: Add an API to read and parse directory metadata

The fact that the uid/gid/mode are big endian bit me when I was
trying to parse this "by hand" in ostree-rs-ext.

Let's add a footgun-free API for this.

(And yeah, we should probably do the same for the other variant types)

ci: add jobs for MSRV checks and linting

This adds two jobs in order to check minimum toolchain compatibility,
and for overall linting.

lib: run rustfmt

lib: fix new clippy warnings

This fixes the following warnings highlighted by clippy:
 * https://rust-lang.github.io/rust-clippy/master/index.html#needless_question_mark
 * https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure
 * https://rust-lang.github.io/rust-clippy/master/index.html#single_component_path_imports

sys/cargo: refresh manifest

This updates stale dependencies and remove leftover settings.

Update impl Display, omit to_string, change wording

add ObjectDetails-struct and use it in list_objects-function

update result type

switch from if-else to match

implement list_objects-function an test

sysroot: support create and load actions on builder

This splits the builder completion step into separate actions for
creating/loading a sysroot.
It also introduces a roundtrip test over a freshly-created empty
sysroot.

sysroot: add a builder object

This adds a `SysrootBuilder` in order to allow consumers to load
a configured `Sysroot` in an ergonomic way. It tries to prevent
logic bugs coming from handling half-initialized entities.

repo: Add `require_rev` method

The `resolve_rev` C method should really have been
`resolve_rev_optional` from the start - it is more obviously wrong
in Rust because the input parameter `allows_noent` controls
whether the returned `Option` can ever be `None`.

I debated adding this to the C bindings, and may still do so,
but eh it's faster to write + ship in Rust, and the future of ostree is
Rust anyways.

(cargo-release) start next development iteration 0.13.4-alpha.0

(cargo-release) version 0.13.3

Bump ostree-sys version

Update to 2021.5

(cargo-release) version 0.13.3-alpha.1

(cargo-release) version 0.13.2

repo: Add `auto_transaction` and `TransactionGuard`

This gives auto-cancelling semantics on `Drop`, plus a nicer
`.commit()` method on the transaction.

Matches the currently private `_OstreeRepoAutoTransaction` in the C
library.

(cargo-release) start next development iteration 0.13.2-alpha.0

(cargo-release) version 0.13.1

repo: Expose dfd_as_file()

The `dfd()` API returns just an integer.  Add a safe API that
makes a copy of the fd.  What we really want here is `BorrowedFd` from
https://github.com/rust-lang/rfcs/blob/master/text/3128-io-safety.md
but that isn't here yet.

Make `SePolicy` have `Send`

It's safe to send between threads, and I want to do so
in ostree-rs-ext to send to a tokio worker thread.

lib: Reexport libc::AT_FDCWD

Useful with `Repo::open_at()`.

Right now ostree-rs-ext pulls in libc for this and `fgetxattr`,
but the latter should go into nix.

sys: Release 0.9.0

Should have been bumped in the previous commit.

Release 0.13

Just an update to support libostree v2021.4, but bumping semver
because a few APIs (correctly) gained `Option<T>`.

Release 0.12.3

No major changes, just exporting the `ffi` bits.

Cargo.toml: Bump to glib 0.14.4

Not strictly required for this repo, but it has the new variant
bindings we want in ostree-rs-ext.

lib: Export ffi too

Matching how gtk-rs does it.  Right now rpm-ostree does depend
on interacting with `ostree-sys` via the cxxrs bits.

Release 0.12.2

```
Colin Walters (8):
      Add 2021.3 feature
      Re-export glib, gio
      Deny unused results, warn on missing docs (except auto/)
      Add new GLib 0.14 variant types for metadata types
      Fix build with --features=v2021_3, use in CI by default
      Add more documentation for --features=v2021_3
      Use glib-sys via re-exported `glib::ffi` (and similar for gio)
      Release 0.12.2

Luca BRUNO (5):
      cargo: fix version in features chain
      lib: fix 'dox' feature
      cargo: point to docs.rs and clean up
      ostree: release 0.12.1
      cargo: bump ostree-sys to 0.8.1
```

Use glib-sys via re-exported `glib::ffi` (and similar for gio)

In general only `-sys` crates should depend on other `-sys`
crates.  IOW for us, `ostree-sys` depends on `glib-sys`.

By using the re-export, we avoid needing to keep a version lock
between `glib` and `glib-sys` in our main crate.  And similar
is true of our higher level reverse dependencies (e.g. `ostree-rs-ext`).

Also weaken our dependency to `0.14` as that's clearer.

Fix build with --features=v2021_3, use in CI by default

It's a huge trap for us not to build with the latest ostree feature
on, I didn't have my IDE configured for it, and CI didn't have
it on.

The previous bump to glib 0.14 broke the Sign code.

Add more documentation for --features=v2021_3

My previous pass was at the default feature level.

Add new GLib 0.14 variant types for metadata types

This way it's more convenient for downstream crates like ostree-rs-ext
to convert loaded variants.

TODO: Can we add a feature for the `gvariant` crate and expose via
that too?

Deny unused results, warn on missing docs (except auto/)

And add basic docs for our manually implemented functions.

cargo: bump ostree-sys to 0.8.1

ostree: release 0.12.1

cargo: point to docs.rs and clean up

This removes stale URLs, pointing to the auto-built docpages at
docs.rs and sorting manifest entries.

lib: fix 'dox' feature

This makes sure docs can be properly built when using the 'dox'
feature. It should fix auto-builds on docs.rs.

cargo: fix version in features chain

This fixes the definition of the `v2016_4` feature.
It restores the chain of versions so that 2016.3 symbols can be
actually reached from newer features/versions.

Re-export glib, gio

Re-export our dependencies.  See https://gtk-rs.org/blog/2021/06/22/new-release.html
"Dependencies are re-exported".  Users will need e.g. `gio::File`, so this avoids
them needing to update matching versions.

Closes: https://github.com/ostreedev/ostree-rs/issues/12

Add 2021.3 feature

(We should add doing this as a SOP for ostree releases)

Bump versions

The glib 0.14 change is semver incompatible.

Update to glib 0.14

An intimidating spam of compiler errors at the start, but the
biggest was handling the new convention of `ostree_sys::` => `ffi::`.

This will require a semver bump of course.

ci: Fix buildroot to use new official image

Which is maintained and has updated rust.

Mark src/auto/* as generated

repo_transaction_stats: move to a manual implementation

This moves `RepoTransactionStats` into a manually implemented
source file in order to provide getters to expose relevant
fields.

Bump sys version requirement